From c8f517940abcfe8fd61e10394621eb8c16651278 Mon Sep 17 00:00:00 2001
From: tsteven4 <13596209+tsteven4@users.noreply.github.com>
Date: Sun, 19 Nov 2023 06:00:31 -0700
Subject: [PATCH] misc cleanups (#1235)

---
 CMakeLists.txt |  1 -
 garmin_gpi.cc  |  2 +-
 tpo.h          | 14 ++------------
 vecs.cc        |  2 +-
 4 files changed, 4 insertions(+), 15 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index b02d70d55..da33def6b 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -256,7 +256,6 @@ set(HEADERS
   jeeps/gpscom.h
   jeeps/gpsdatum.h
   jeeps/gpsdevice.h
-  jeeps/gpsfmt.h
   jeeps/gpsmath.h
   jeeps/gpsmem.h
   jeeps/gpsport.h
diff --git a/garmin_gpi.cc b/garmin_gpi.cc
index 3f9a824fb..a936cd373 100644
--- a/garmin_gpi.cc
+++ b/garmin_gpi.cc
@@ -452,7 +452,7 @@ GarminGPIFormat::read_tag(const char* caller, const int tag, Waypoint* wpt)
 
   case 0x8000b:  /* address (street/city...) */
     (void) gbfgetint32(fin);
-  // FALLTHROUGH
+    [[fallthrough]];
   case 0xb:  /* as seen in German POI files. */
     PP;
     mask = gbfgetint16(fin); /* address fields mask */
diff --git a/tpo.h b/tpo.h
index 43a0c7e40..ddcf9ebc9 100644
--- a/tpo.h
+++ b/tpo.h
@@ -138,7 +138,7 @@ class Tpo2Format : public Format, private TpoFormatBase
 public:
   QVector<arglist_t>* get_args() override
   {
-    return &tpo2_args;
+    return nullptr;
   }
 
   ff_type get_type() const override
@@ -165,11 +165,6 @@ public:
   {
     tpo_rd_deinit();
   }
-
-private:
-  /* Data Members */
-
-  QVector<arglist_t> tpo2_args = {};
 };
 
 class Tpo3Format : public Format, private TpoFormatBase
@@ -177,7 +172,7 @@ class Tpo3Format : public Format, private TpoFormatBase
 public:
   QVector<arglist_t>* get_args() override
   {
-    return &tpo3_args;
+    return nullptr;
   }
 
   ff_type get_type() const override
@@ -204,10 +199,5 @@ public:
   {
     tpo_rd_deinit();
   }
-
-private:
-  /* Data Members */
-
-  QVector<arglist_t> tpo3_args = {};
 };
 #endif // TPO_H_INCLUDED_
diff --git a/vecs.cc b/vecs.cc
index 42c1f4291..1a232f408 100644
--- a/vecs.cc
+++ b/vecs.cc
@@ -706,7 +706,7 @@ void Vecs::disp_vec_options(const QString& vecname, const QVector<arglist_t>* ar
 {
   if (args) {
     for (const auto& arg : *args) {
-      if (*arg.argval && arg.argval) {
+      if (arg.argval && *arg.argval) {
         printf("options: module/option=value: %s/%s=\"%s\"",
                qPrintable(vecname), qPrintable(arg.argstring), *arg.argval);
         if (case_ignore_strcmp(arg.defaultvalue, *arg.argval) == 0) {
-- 
2.30.2